New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated developer documenation #2951
Conversation
Hello @nabobalis! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2019-03-20 21:03:11 UTC |
Thanks for the pull request @nabobalis! Everything looks great! |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wont pretend to have gone through all this line-by-line but I think it's easy to see it's an improvement on what we have at the moment. I think the contributing.rst is very good (that I did read all of).
* Added another section to the newcomers guide on checking your code. * Addressed comments about confusing sections. * Fixed some typos.
I'm gonna merge this in now and if any problems arise, we can fix them after. |
Description
This is part 1 of the epic journey that is, "Nabil goes over all the files in SunPy".
Merged in #2934
Fixes #2931
Fixes #2925
Fixes #1858
Fixes #856
Fixes #1618
Fixes #212
CONTRIBUTING.rst is the one to check the most
The other changes are mainly within the
dev_guide
section that hopefully explain more about our test suite and how to write/run aspects of SunPyBuilt docs.